Skip to content

Remove generated Terraform docs from infrastructure#814

Merged
silvicir merged 2 commits into
masterfrom
chore/remove-generated-terraform-docs
Jul 6, 2026
Merged

Remove generated Terraform docs from infrastructure#814
silvicir merged 2 commits into
masterfrom
chore/remove-generated-terraform-docs

Conversation

@krusty93

@krusty93 krusty93 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR removes generated Terraform docs from infrastructure repositories where README files are not useful documentation.

Changes

  • Remove terraform_docs from the root pre-commit configuration.
  • Remove generated <!-- BEGIN_TF_DOCS -->...<!-- END_TF_DOCS --> blocks from infra/**/README.md.
  • Delete README files that contained only generated Terraform docs or only a heading plus generated Terraform docs.

@krusty93 krusty93 requested review from a team as code owners July 2, 2026 14:17
@krusty93 krusty93 requested review from grausof and silvicir and removed request for a team July 2, 2026 14:17
@github-actions github-actions Bot added the infra label Jul 2, 2026
@krusty93 krusty93 temporarily deployed to infra-prod-ci July 2, 2026 14:18 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Warning

⚠️ Trivy detected forbidden Terraform patterns

The following findings must be resolved before this check becomes enforced.

Severity ID File Line Title
HIGH AVD-GIT-0002 infra/bootstrapper/prod/secrets.tf L233 Ensure plaintext value is not used for GitHub Action Environment Secret.
HIGH AVD-GIT-0002 infra/bootstrapper/prod/secrets.tf L240 Ensure plaintext value is not used for GitHub Action Environment Secret.
HIGH AVD-GIT-0002 infra/bootstrapper/prod/secrets.tf L251 Ensure plaintext value is not used for GitHub Action Environment Secret.
HIGH AVD-DX-0001 infra/resources/prod/data.tf L43 Terraform must not read Key Vault secrets via data source
HIGH AVD-DX-0001 infra/resources/prod/data.tf L38 Terraform must not read Key Vault secrets via data source
HIGH AVD-DX-0001 infra/resources/psn/prod/apim.tf L1 Terraform must not read Key Vault secrets via data source
HIGH AVD-DX-0001 infra/resources/psn/prod/data.tf L157 Terraform must not read Key Vault secrets via data source
HIGH AVD-DX-0001 infra/resources/psn/prod/data.tf L152 Terraform must not read Key Vault secrets via data source

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Tip

✅ All Terraform module locks are up to date

No module changes detected - everything is in sync!

📋 Pre-commit Output Log 
[INFO] Initializing environment for https://github.com/pagopa/dx.
[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
Lock Terraform Registry modules..........................................Passed

Generated on Thu Jul 2 14:26:57 UTC 2026
Run lock_modules on folder: infra/resources/prod/

@krusty93 krusty93 temporarily deployed to infra-prod-ci July 2, 2026 14:27 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

📖 Terraform Plan (infra/resources/prod) - success

Show Plan 
No changes detected.

@silvicir silvicir merged commit fcb95ef into master Jul 6, 2026
26 checks passed
@silvicir silvicir deleted the chore/remove-generated-terraform-docs branch July 6, 2026 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silvicir silvicir silvicir approved these changes
@grausof grausof Awaiting requested review from grausof grausof is a code owner automatically assigned from pagopa/io-wallet

Assignees

No one assigned

Labels

infra

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@krusty93 @silvicir